Conversation
|
#39 is not merge-ready. The custom Transformers v5 loading path is healthy, but exact-head testing on the proposed dependency stack found model-correctness failures and broken release gates. Blocking findings
Additional actionable problems
Positive resultOn the requested canonical environment:
Offline
So the earlier Transformers v5 custom-class/weight-loading concern appears resolved. It does not offset the ESMC parity, ESMFold2 FP8, offline kernel, and provenance blockers above. Recommendation: request changes. At minimum, fix or remove the failing experimental ESMFold2 FP8 artifact, restore all ESMC contract/engineering gates, repair locked offline kernel loading on the proposed versions, and correct the embedding/artifact provenance issues before merging. |
PR 39 merge-readiness implementation and validation reportRecommendationThis work materially improves PR 39, and the ESMFold2 precision policy is now Validation was performed from a working tree based on PR head What changedModel correctness and Transformers 5 compatibility
ESMFold2 policy and biological validation
Runtime and shared APIs
Artifacts, offline kernels, packaging, and release automation
Locked H100 validation environment
Test resultsFocused and CPU validation
The three full-CPU failures are known artifact-golden availability or stale Transformers 5.13 and artifact loading
ESMC BF16 calibrationThe calibration covered all three ESM++ sizes, all five attention backends, a The predeclared ESMFold2 real-protein panelThe panel produced 60 official BF16 bundles, 60 FastPLMs BF16 bundles, and 60 FastPLMs BF16 matched official BF16 in all 60 cases: lDDT-C-alpha was
The complete assertion gate passed 43 of 60 cases. Eight absolute-geometry
ipTM error was zero in every case. The widest FP8 deviations occurred for Artifact and aggregate release status
Remaining merge requirements
Until those items are complete, the PR should remain open. |
PR 39 merge-readiness updateCurrent PR head: Implemented changes
The 60-case ESMFold2 FP8 parity panel and advisory workflow were not included. Workstation resultsAll execution was performed on the designated workstation.
Exact-head release statusThe aggregate release attempt successfully built all 29 artifacts, then failed in the offline artifact phase because exact checkpoint-key parity was incorrectly being required for alternate task heads. For example, loading an MLM checkpoint through a sequence-classification AutoClass necessarily has a newly initialized classifier and an unused MLM head. That release-gate bug is fixed in The complete aggregate suite was not rerun after An approving review is also still required before merge. |
Close parity and packaging gaps found during PR review. Pin compact official golden bundles for every check-tier model.
Keep Flex Attention and FlashAttention 3 selectable while documenting the known ESMC-6B BF16 boundary deviations. Track those cases as strict expected failures so improved kernels force the limitation to be revisited.
where
H: (b, l, 81, 2560)producesZ: (b, l, 256).base_z_linear,base_z_combine, and checkpoint parameter keys.ESMFold2 FP8
esmc_precision="auto" | "bf16" | "fp32" | "fp8".reload_esmc(...)and structured precision status.Other models and feature paths
Containers and remote validation
ipc: host, cache volumes, source mounting, and output directories.Local Hub artifacts
sys.path.Testing and compliance
tests/unittests/integrationtests/paritytests/structuretests/releaseBenchmarking
Documentation
Validation environment
All verification was run on the supplied H100 workstation using:
flash-attnValidation results
ESM++ and ESMC
ESMFold2
Artifacts and release contracts
--check.git diff --checkpasses.Known boundaries and release handoff
dist/hubartifacts are build outputs and are not committed.0.01to0.012on one seeded random compliance batch. This is documented as reproducibility context rather than a general biological-sequence error estimate.Review guidance
The highest-value review surfaces are:
src/fastplms/models.tomlandsrc/fastplms/registry.pyfor the authoritative support and provenance contract.src/fastplms/attention/andkernels.lockfor backend selection, masking, and precompiled FlashAttention policy.src/fastplms/embeddings/for residue selection, pooling, persistence, and resume behavior.src/fastplms/models/esm_plusplus/for ESMC architecture and exact SDPA behavior.src/fastplms/models/esmfold2/for the learned representation and precision/reload policy.tests/parity/,tests/structure/, andtests/release/for the compliance and release gates.docker/,tools/artifacts/, andtools/remote/for reproducible validation and distribution.